Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrading smartcontract #735

Merged
merged 25 commits into from
Jul 10, 2024
Merged

Conversation

KodeSage
Copy link
Contributor

@KodeSage KodeSage commented Jun 28, 2024

Bounty Submission for #626

Hello @briwylde08 This is ready for review

@briwylde08
Copy link
Contributor

Thank you so much! We'll get it reviewed and let you know if there are any questions or follow-ups!

@briwylde08 briwylde08 requested a review from ElliotFriend July 1, 2024 17:32
ElliotFriend and others added 16 commits July 1, 2024 14:16
* remove unnecessary slash from a horizon redirect

* reorder redirects for api references

* fix redirects for hubble

* fix some redirects for RPC
* fix nginx config.

* Update storage language

---------

Co-authored-by: overcat <[email protected]>
* fix nginx config.

* Docs - Build Tab

* Moving things around

* moving some more things around

* moving more things around

* messing with tokens

* moving and shuffling some files around

* moving some more things from the most recent `main` commits

* add a TODO about re-setting broken links configs

* shifting around token things

* update token interface to v20.5.0

* Adjustments

* start on fixing the broken linkies

* link extravaganza

* links

* links

* content updates

* content updates and formatting

* homepage

* redirects

* fix some broken links in asset issuing tutorial

* fix learn URL on index page

* moving and shuffling build/token redirects

* rename dapp frontend filename

* adjusting/tweaking some older redirects

* links

* add a couple more redirect changes

* specify the next/prev pagination items since we moved create frontend

* /docs content

* Moving assets and tokens

* links

* use mdx links in docs/README.mdx

* misc fixes in stellar.toml info page

* remove unnecessary heading in tokens quickstart page

* shuffle and reorg sidebar declarations (no affect on actual docs)

* expand smart contracts section in build sidebar

* add some frontmatter config to docs/README.mdx

* re-adding redirects that were overwritten

---------

Co-authored-by: overcat <[email protected]>
Co-authored-by: Elliot Voris <[email protected]>
* Add stellar wallet kit to developer tools

* Add link to docs
* format, check

* file name change

* fix confusing file paths

* added the complete script
* Data Introduction (draft)

* Stellar RPC -> RPC
* fix nginx config.

* Add descriptions to How-To Guide boxes

* more descriptions

---------

Co-authored-by: overcat <[email protected]>
* fix some broken, old links caught by the algolia crawler

* re-generate specfile docs from changed files

* broken link in anchor platform
@KodeSage
Copy link
Contributor Author

KodeSage commented Jul 3, 2024

Hi @ElliotFriend @briwylde08 , I have resolved the merge conflict and updated file folder structure.

Updated File Path that I worked on : docs/build/guides/conventions/upgrading-contracts.mdx

@briwylde08
Copy link
Contributor

Thanks for fixing the conflicts! I'm guessing we'll get this reviewed next week. It's a holiday tomorrow in the US so most of the team has a short week. 🙏

@KodeSage
Copy link
Contributor Author

KodeSage commented Jul 9, 2024

@briwylde08 alright, looking forward to it.

Copy link
Contributor

@ElliotFriend ElliotFriend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, there @KodeSage!! Looks really good! I left a few suggestions/changes throughout the file, and some of them are only nitpicks.

Thanks again!

docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
@ElliotFriend
Copy link
Contributor

Also, I don't know why Github is telling us that 243 files have been changed 🤣 I can see by comparing here that it's really only the one file. So... 🤷🏻‍♂️

@KodeSage
Copy link
Contributor Author

KodeSage commented Jul 9, 2024

Hi @ElliotFriend , I will definitely make up this changes and I push up soon.

For the 243 files changed 😂, I have worked on this tasks before the changes you guys made in the codebase, so I had to pull all those changes in the branches I'm working on the different tasks assigned to me.

@KodeSage
Copy link
Contributor Author

KodeSage commented Jul 9, 2024

HI @ElliotFriend , I made changes to the file you reviewed and I have pushed the updated format. please re-review.

@KodeSage KodeSage requested a review from ElliotFriend July 10, 2024 09:29
Copy link
Contributor

@ElliotFriend ElliotFriend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the changes, @KodeSage! I left a few more comments, almost entirely nitpicks. I think this is really close to ready!

docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
docs/build/guides/conventions/upgrading-contracts.mdx Outdated Show resolved Hide resolved
@ElliotFriend
Copy link
Contributor

I'm also noticing that this commit also includes the file from your other PR (#737). Can you remove it from here, so we don't end up with conflicting files?

@KodeSage
Copy link
Contributor Author

@ElliotFriend I will make changes to them now and push up in few minutes time.

@KodeSage
Copy link
Contributor Author

KodeSage commented Jul 10, 2024

Hi @ElliotFriend , I made a push now with the updated changes please review.

@ElliotFriend
Copy link
Contributor

@KodeSage Looks good! Once the build workflow finishes successfully, I'll give an approval.

Copy link
Contributor

@ElliotFriend ElliotFriend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @KodeSage! Good work!!

@briwylde08
Copy link
Contributor

Thank you so much for your work here @KodeSage! I'll merge this and we can take the award conversation back to the corresponding issue.

@briwylde08 briwylde08 merged commit 538e603 into stellar:main Jul 10, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants